Skip to content

Feature/cdi previewer#133

Merged
qqmyers merged 168 commits intogdcc:developfrom
ErykKul:feature/cdi-previewer
Nov 28, 2025
Merged

Feature/cdi previewer#133
qqmyers merged 168 commits intogdcc:developfrom
ErykKul:feature/cdi-previewer

Conversation

@ErykKul
Copy link
Copy Markdown

@ErykKul ErykKul commented Nov 26, 2025

This PR adds the CDI previewer to previewers/betatest and includes:

  • the bundled cdi-viewer (inlined JS bundle) and a minimal embedding page (CdiPreview.html)
  • SHACL shapes used for validation and suggestions (cdif-core.ttl, ddi-cdi-official.ttl + jsonld)
  • small README link to the standalone implementation repository: https://github.com/libis/cdi-viewer

ErykKul and others added 24 commits November 19, 2025 00:40
…state and enable save button

feat(event-handlers): collect changes before toggling edit mode
…er Bundle

- Updated the SHACL shapes in cdif-core.ttl to use SPARQL targets for better validation of Dataset nodes.
- Simplified property definitions and added comments for clarity.
- Introduced a new HTML file (test-cdi-bundle.html) to test the functionality of the CDI Viewer Bundle, including validation and loading of test CDI files.
- Enhanced user interface with Bootstrap for better presentation of test results.
Copy link
Copy Markdown
Member

@qqmyers qqmyers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Looks fine with one minor request - move the curl commands to the end of the commands.md docs into the (currently empty) betatest section - since the previewer only exists in the /betatest branch at this point. (

@ErykKul
Copy link
Copy Markdown
Author

ErykKul commented Nov 28, 2025

I moved the commands in 6.1 and removed them in 5.2, as support for that is not implemented in this previewer.

@qqmyers qqmyers merged commit c037535 into gdcc:develop Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants